ci(review-debt-guard): 빚의 기준을 «리뷰어의 공이 남은 PR» 로 [base: develop] - #1148

리뷰 요청은 알림 수단이지 담당 배정이 아니다. 누구에게 걸렸든 팀에서 한 사람이
리뷰하면 그 PR 은 끝난 것이고, 반대로 아무도 응답하지 않은 PR 이 남아 있으면
요청을 받았든 아니든 리뷰를 미룬 것이다.

그래서 판정 대상을 review-requested 검색이 아니라 열린 PR 전체로 바꾼다.
자기 PR 은 자신이 리뷰할 수 없으므로 빼고, draft 는 리뷰받을 상태가 아니므로 뺀다.

실측(로컬 드라이런, close 직전까지):

    koongmai      22
    Sadturtleman  20
    1hyok          2

열린 non-draft PR 48건 중 아무도 응답하지 않은 것이 22건이고, 그중 20건이
1hyok 의 PR 이라 사람마다 빚이 다르게 잡힌다.

승인 뒤에 작성자가 손보는 것은 작성자 몫이고, 그 변경을 다시 봐 줄 의무는
누구에게도 없다. 리뷰가 다시 필요하면 작성자가 요청을 걸면 된다.

그래서 «마지막 실질 변경 이후» 라는 시점 조건을 걷어낸다. 판정
(APPROVED·CHANGES_REQUESTED)이 한 번이라도 나온 PR 은 빚에서 제외한다.

커밋 시각을 볼 일이 없어져 parents 조회가 통째로 빠지고, PR 당 API 가 2회에서
1회로 준다. 가드 실행도 그만큼 짧아진다.

실측(로컬 드라이런, close 직전까지):

    koongmai      4
    Sadturtleman  4
    1hyok         0 (통과)

열린 non-draft PR 48건 중 판정이 한 번도 없는 것은 #1133 #1134 #1138 #1139
넷뿐이고 모두 1hyok 의 PR 이라, 두 사람의 빚이 각각 4건으로 잡힌다.

승인과 변경요청의 무게가 다르다.

  APPROVED           머지 가능한 상태다. 그 뒤 작성자가 손보는 것은 작성자 몫이고
                     다시 봐 줄 의무는 누구에게도 없다.
  CHANGES_REQUESTED  끝난 판정이 아니다. 작성자가 지적을 반영하면 공이 리뷰어에게
                     돌아온다. 재리뷰가 없으면 그 PR 은 영영 머지되지 못한다.

직전 커밋은 둘을 «판정 1건» 으로 묶어, 변경요청을 받고 반영까지 끝낸 #821 이
빚에서 빠졌다. 그 상태로는 아무도 재리뷰할 의무가 없어 PR 이 갇힌다.

그래서 빚을 두 종류로 나눈다.

  1. 아무도 판정을 내지 않은 PR
  2. 미해소 변경요청이 있는데 작성자가 이미 반영 커밋을 올린 PR

변경요청 뒤로 새 커밋이 없으면 공은 아직 작성자에게 있으므로 빚이 아니다.
반영 여부는 병합 커밋(parents 2개 이상)을 뺀 실질 커밋으로만 센다.

실측(로컬 드라이런, close 직전까지):

    koongmai      11
    Sadturtleman  11
    1hyok          0 (통과)

    #1133  아무도 판정하지 않음
    #966   승인됨, 빚 아님
    #821   변경요청 반영(4커밋) 뒤 재리뷰 없음
    #958   변경요청 미반영, 공은 작성자에게